Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix controllers index #664

Merged

Conversation

christophfroehlich
Copy link
Contributor

Sorry, there was a remnant of the merge conflict by #661

Copy link
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder how many times we all fixed those by now and broke it again in unexpected ways :)

@bmagyar bmagyar merged commit cb7c455 into ros-controls:humble Jun 12, 2023
@christophfroehlich christophfroehlich deleted the fix_controllers_index branch June 12, 2023 15:41
@christophfroehlich
Copy link
Contributor Author

maybe we can add a CI job for the docs in the main repos? ;)

@destogl
Copy link
Member

destogl commented Jun 12, 2023

maybe we can add a CI job for the docs in the main repos? ;)

We probably should...

@destogl
Copy link
Member

destogl commented Jun 12, 2023

We can actually use external workflows if we convert one for the checks of the documentation into reusable workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants